Skip to content

build: Copy setup.cfg data to pyproject.toml#886

Open
pwithnall wants to merge 1 commit into
masterfrom
pyproject-toml
Open

build: Copy setup.cfg data to pyproject.toml#886
pwithnall wants to merge 1 commit into
masterfrom
pyproject-toml

Conversation

@pwithnall
Copy link
Copy Markdown
Contributor

This is needed to use newer versions of kolibri-tools, which will read configuration from pyproject.toml in preference to setup.cfg, if the former exists.

See learningequality/kolibri#11146

For now, keep the old configuration present in setup.cfg as well, in case people are using an older version of kolibri-tools. That can be dropped once we bump our dependency.

This is needed to use newer versions of kolibri-tools, which will read
configuration from `pyproject.toml` in preference to `setup.cfg`, if the
former exists.

See learningequality/kolibri#11146

For now, keep the old configuration present in `setup.cfg` as well, in
case people are using an older version of kolibri-tools. That can be
dropped once we bump our dependency.

Signed-off-by: Philip Withnall <pwithnall@endlessos.org>
@pwithnall pwithnall requested a review from dbnicholson October 19, 2023 11:28
@pwithnall pwithnall self-assigned this Oct 19, 2023
@pwithnall
Copy link
Copy Markdown
Contributor Author

This is actually only needed once kolibri-explore-plugin depends on a new-enough version of kolibri-tools that contains learningequality/kolibri#11146. At that point, our i18n would break without this PR, because kolibri-tools will read pyproject.toml in preference to setup.cfg, and won’t merge keys between the two.

Submitting this PR now, even though it’s not actually needed yet, to avoid that potential trip hazard in future.

@manuq
Copy link
Copy Markdown
Collaborator

manuq commented Oct 19, 2023

Thank you @pwithnall

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants